Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add In-App Forms to README #199

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Add In-App Forms to README #199

wants to merge 5 commits into from

Conversation

belleklaviyo
Copy link
Contributor

@belleklaviyo belleklaviyo commented Mar 18, 2025

Description

https://klaviyo.atlassian.net/browse/CHNL-16763

Read the formatted README here

@dan-peluso
Copy link
Contributor

I think we're going to need some sort of section on setup for Android - we need to let them know that they have to call

Applcation.onCreate() {
    Klaviyo.registerForLifecycleCallbacks(this)
}

in the native layer (unless they are initializing the Klaviyo SDK from the native layer in the Application onCreate) or else in app forms won't work. @evan-masseau any thoughts on that copy? Definitely want to make it obvious that this is required on Android.

Copy link

@mark-piana mark-piana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants